Add Portuguese (Brazil) localization strings - #109
Conversation
values-pt-rBR "Please, add language 'pt-br'"
There was a problem hiding this comment.
🟡 Changes recommended
The pt-BR strings file includes multiple locale-only keys not present in the default values/strings.xml, which can lead to missing-fallback runtime failures if those keys are ever referenced.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Adds a new Android resource set for Brazilian Portuguese (pt-BR) so the app can display localized UI strings when the device locale matches.
Changes:
- Added
values-pt-rBR/strings.xmlwith pt-BR translations for existing UI labels, actions, messages, counters, and error strings. - Introduced additional pt-BR-only string keys (e.g., language/UI settings related entries) that are not present in default resources.
File summaries
| File | Description |
|---|---|
| app/src/main/res/values-pt-rBR/strings.xml | Adds pt-BR string resources for the app UI. |
Review details
Suppressed comments (1)
app/src/main/res/values-pt-rBR/strings.xml:103
error_link_title_smallis defined only in this pt-BR file; it is not present invalues/strings.xml(default) or any other locale, and it is not referenced inapp/src/main. If this string is intended to be used, it should be added to the default strings so other locales have a fallback; otherwise, consider removing it to avoid accumulating orphaned resources.
<string name="error_link_title_empty">O título do link não pode ficar vazio</string>
<string name="error_link_title_small">O título do link deve ter pelo menos 3 caracteres</string>
<string name="error_link_subtitle_empty">O subtítulo do link não pode ficar vazio</string>
- Files reviewed: 1/1 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| <string name="language">Idioma</string> | ||
| <string name="system_default">Padrão do sistema</string> | ||
| <string name="portuguese_brazil">Português (Brasil)</string> | ||
| <string name="app_info">Informações do app</string> | ||
| <string name="ui_settings">Interface</string> |
|
Hello @marcositz, Thank you for adding the br language, but with only this change, it will not work, |
values-pt-rBR "Please, add language 'pt-br'"